update mcp-gateway authn/authz guides#110677
update mcp-gateway authn/authz guides#110677Patryk-Stefanski wants to merge 5 commits intoopenshift:mcp-gateway-docs-tpfrom
Conversation
openshift-ci
Bot
added
the
size/M
|
🤖 Thu Apr 23 23:17:12 - Prow CI generated the docs preview: https://110677--ocpdocs-pr.netlify.app/rhcl/latest/mcp_gateway_config/mcp-gateway-authentication.html |
Signed-off-by: Patryk Stefanski <pstefans@redhat.com>
Patryk-Stefanski
force-pushed
the
fix-auth-authz-docs
branch
from
2f17d5e to
cd1c260
Compare
| + | ||
| [source,json,subs="+quotes"] | ||
| ---- | ||
| $ oc patch gateway _<mcp_gateway>_ -n _<gateway_system>_ --type json -p '[ |
There was a problem hiding this comment.
should this section, lines 44-50 in the install doc be updated to match this?
- we can make the listener a prereq to this procedure
- add a note to the install doc that you need to config this listener to create authorization
- then we can remove the step from this procedure
WDYT?
There was a problem hiding this comment.
sounds good to me, I removed it from this pr and added it as a prereq. Do you want to add it to your current installation PR, or will I create a follow-up PR?
| ---- | ||
| $ oc apply -f _<mcp_tool_auth_policy.yaml>_ | ||
| $ oc apply -f - <<EOF | ||
| <AuthPolicy CR from the previous step> |
There was a problem hiding this comment.
| <AuthPolicy CR from the previous step> | |
| _<mcp_tool_auth_policy.yaml>_ |
user-replaceable values can only look like this
I also don't think the EOF works when we separate file creation from application? (having the yaml in one step and application of the resource in a second step is preferred in docs)
Co-authored-by: Shauna Diaz <shdiaz@redhat.com>
Co-authored-by: Shauna Diaz <shdiaz@redhat.com>
Co-authored-by: Shauna Diaz <shdiaz@redhat.com>
…n doc Signed-off-by: Patryk Stefanski <pstefans@redhat.com>
Patryk-Stefanski
force-pushed
the
fix-auth-authz-docs
branch
from
f224922 to
7ce29ce
Compare
| + | ||
| Replace `_<mcp_jwt_authpolicy.yaml>_` with the name of your CR. | ||
|
|
||
| Replace `_<mcp_jwt_auth_policy.yaml>_` with the name of your CR. |
There was a problem hiding this comment.
🤖 [error] AsciiDocDITA.TaskStep: Content other than a single list cannot be mapped to DITA steps.
|
@Patryk-Stefanski: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
4 participants
Version(s):
mcp-gateway-docs-tp
Issue:
OSDOCS-17719
OSDOCS-17718
Link to docs preview:
https://110677--ocpdocs-pr.netlify.app/rhcl/latest/mcp_gateway_config/mcp-gateway-authentication.html
https://110677--ocpdocs-pr.netlify.app/rhcl/latest/mcp_gateway_config/mcp-gateway-authorization.html
QE review:
Additional information:
See #109794 and #109672 for context.